gtk: fix tool palette scroll adjustment setting
authorMichael Natterer <mitch@gimp.org>
Fri, 22 Oct 2010 15:01:58 +0000 (17:01 +0200)
committerMichael Natterer <mitch@gimp.org>
Fri, 22 Oct 2010 15:01:58 +0000 (17:01 +0200)
Create a new adjustment if there was *no* adjustment passed in, not
the other way around. Also remove unused function
gtk_tool_palette_set_adjustment().

gtk/gtktoolpalette.c

index 0359fcfc45fc45f5f51682a1977447c2372c874f..fa606197a2a4675c88fe382ca6c12f9d3a80988a 100644 (file)
@@ -750,50 +750,6 @@ gtk_tool_palette_adjustment_value_changed (GtkAdjustment *adjustment,
   gtk_tool_palette_size_allocate (widget, &allocation);
 }
 
-static void
-gtk_tool_palette_set_adjustment (GtkToolPalette *palette,
-                                GtkOrientation  orientation,
-                                GtkAdjustment  *adjustment)
-{
-  GtkAdjustment **adj_ptr;
-
-  if (orientation == GTK_ORIENTATION_HORIZONTAL)
-    {
-      if (adjustment && palette->priv->hadjustment == adjustment)
-       return;
-
-      adj_ptr = &palette->priv->hadjustment;
-    }
-  else
-    {
-      if (adjustment && palette->priv->vadjustment == adjustment)
-       return;
-
-      adj_ptr = &palette->priv->vadjustment;
-    }
-
-  /* Disconnect handler */
-  if (*adj_ptr)
-    {
-      g_signal_handlers_disconnect_by_func (*adj_ptr,
-                                           gtk_tool_palette_adjustment_value_changed,
-                                           palette);
-      g_object_unref (*adj_ptr);
-    }
-
-  /* Ensure adjustment */
-  if (!adjustment)
-    adjustment = GTK_ADJUSTMENT (gtk_adjustment_new (0.0, 0.0, 0.0,
-                                                    0.0, 0.0, 0.0));
-
-  /* Connect signal handler */
-  g_signal_connect (adjustment, "value-changed",
-                   G_CALLBACK (gtk_tool_palette_adjustment_value_changed),
-                   palette);
-  *adj_ptr = g_object_ref_sink (adjustment);
-  /* FIXME: Adjustment should probably have it's values updated now */
-}
-
 static void
 gtk_tool_palette_add (GtkContainer *container,
                       GtkWidget    *child)
@@ -1977,7 +1933,7 @@ gtk_tool_palette_set_hadjustment (GtkToolPalette *palette,
       g_object_unref (priv->hadjustment);
     }
 
-  if (adjustment != NULL)
+  if (adjustment == NULL)
     adjustment = gtk_adjustment_new (0.0, 0.0, 0.0,
                                      0.0, 0.0, 0.0);
 
@@ -2026,7 +1982,7 @@ gtk_tool_palette_set_vadjustment (GtkToolPalette *palette,
       g_object_unref (priv->vadjustment);
     }
 
-  if (adjustment != NULL)
+  if (adjustment == NULL)
     adjustment = gtk_adjustment_new (0.0, 0.0, 0.0,
                                      0.0, 0.0, 0.0);